Conversation
|
I fixed the lint errors, would you mind pulling the PR? |
|
Apologies for multiple failing lint commits. Im sure its correct now. |
Hi thanks for this PR. Current code: What do you think to do it like this?: This ensures buttons are tracked even if Masque loads after MultiBot. Otherwise, the late-loaded Masque won’t skin already-created buttons. In Core/MultiBotHandler.lua Current code What do you think to do it like this?: If Masque is loaded after MultiBot, we need a hook to trigger initialization and apply the skin to already-created buttons. |
|
I never considered that, nice fail save! |
|
@Wishmaster117 I added your solution to the PR |
Hi sorry to be so long but I'm working in 3 projects lol, i try to test that tomorrow and merge. |
I added code to allow for Masque button support. I'd love to be able to style my buttons ^^. Surely most of you alll have the same feeling. Hopefully you'll accept my PR.
This is the Masque addon: https://github.com/bkader/Masque-WoTLK